Skip to content

Fix workspace header offset and clarify snapshot copy#248

Merged
Cataldir merged 1 commit into
mainfrom
fix/workspace-header-offset-copy
May 21, 2026
Merged

Fix workspace header offset and clarify snapshot copy#248
Cataldir merged 1 commit into
mainfrom
fix/workspace-header-offset-copy

Conversation

@Cataldir
Copy link
Copy Markdown
Contributor

Summary

  • measures the fixed workspace header and mirrors its height into the shell offset so sidebar/content no longer render under the header
  • keeps mobile/tablet sidebar behavior and desktop sidebar visibility intact
  • replaces internal backend snapshot contract wording with user-facing Insights workspace snapshot copy
  • adds route-shell assertions that workspace sidebar and main panel start below the fixed header

Source of the displayed snapshot text

The admin headline comes from the Insights /workspace-snapshots/admin response (snapshot.summary). The admin summary is generated in apps/insights/src/app/projections.py from deterministic seeded operational counts for the selected context. The explanatory paragraph is frontend copy in WorkspaceHome.

Validation

  • pnpm --dir frontend typecheck
  • pnpm --dir frontend lint
  • pnpm --dir frontend exec playwright test e2e/route-shells.spec.ts
  • git diff --check

@Cataldir Cataldir merged commit 54287f4 into main May 21, 2026
10 checks passed
@Cataldir Cataldir deleted the fix/workspace-header-offset-copy branch May 21, 2026 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant